Skip to content

feat: refine entrypoint selection logic#181

Open
radugheo wants to merge 1 commit intomainfrom
fix/add-single-entrypoint-check
Open

feat: refine entrypoint selection logic#181
radugheo wants to merge 1 commit intomainfrom
fix/add-single-entrypoint-check

Conversation

@radugheo
Copy link
Collaborator

@radugheo radugheo commented Feb 25, 2026

  • removed fallback logic in get_server() so that server lookup now strictly uses the provided name instead of returning the only available server if only one exists
  • updated uipath to enable entrypoints autodiscovery (related to feat: add entrypoint autodiscover uipath-python#1431)

to do:

  • replace uipath dev version with a stable version

@radugheo radugheo changed the title Fix/add single entrypoint check 🚧fix/add single entrypoint check Feb 25, 2026
@radugheo radugheo changed the base branch from feat/add-token-refresh-mechanism to main February 25, 2026 11:53
@radugheo radugheo force-pushed the fix/add-single-entrypoint-check branch 4 times, most recently from 37ea81a to 9d54c8b Compare March 11, 2026 09:37
@radugheo radugheo changed the title 🚧fix/add single entrypoint check fix/add single entrypoint check Mar 11, 2026
@radugheo radugheo marked this pull request as ready for review March 11, 2026 09:40
@radugheo radugheo requested review from Copilot and edis-uipath March 11, 2026 09:40
@radugheo radugheo changed the title fix/add single entrypoint check fix: add single entrypoint check Mar 11, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR tightens MCP server selection behavior to require an explicit matching server name (entrypoint) and updates dependencies to a newer uipath build intended to support entrypoint autodiscovery.

Changes:

  • Removed the “single server fallback” behavior in McpConfig.get_server() so lookup is strictly by name.
  • Updated dependency versions (notably uipath, uipath-runtime, uipath-core) and lockfile entries; added UV source configuration for TestPyPI.
  • Ignored a root-level .claude/ directory in .gitignore.

Reviewed changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated 1 comment.

File Description
uv.lock Updates locked versions and switches uipath resolution to TestPyPI dev build; adds uipath-platform dependency.
src/uipath_mcp/_cli/_utils/_config.py Makes get_server(name) strictly return only the named server (no single-server fallback).
pyproject.toml Pins uipath to a dev version and configures UV to pull it from TestPyPI.
.gitignore Adds .claude/ at repo root to ignored paths.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

pyproject.toml Outdated
Comment on lines +10 to +11
# "uipath>=2.8.23, <2.9.0",
"uipath==2.10.12.dev1014315279",
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

uipath is pinned to a TestPyPI-only dev build (2.10.12.dev1014315279). This will make pip install uipath-mcp fail for end users because pip won’t consult [tool.uv.*] indexes/sources and won’t find that version on PyPI. Please switch back to a stable PyPI release before merging, or (if this must remain temporarily) move this to a dev-only dependency path and/or use a PEP 508 direct URL dependency so non-uv installers can resolve it deterministically.

Suggested change
# "uipath>=2.8.23, <2.9.0",
"uipath==2.10.12.dev1014315279",
"uipath>=2.8.23, <2.9.0",
# "uipath==2.10.12.dev1014315279",

Copilot uses AI. Check for mistakes.
@radugheo radugheo force-pushed the fix/add-single-entrypoint-check branch from 09ae039 to ea28f73 Compare March 11, 2026 12:45
@radugheo radugheo changed the title fix: add single entrypoint check feat: refine entrypoint selection logic Mar 11, 2026
@radugheo radugheo force-pushed the fix/add-single-entrypoint-check branch 2 times, most recently from ea084be to 6e6afd4 Compare March 11, 2026 13:04
@radugheo radugheo force-pushed the fix/add-single-entrypoint-check branch from 6e6afd4 to 51539eb Compare March 11, 2026 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants